-
-
Notifications
You must be signed in to change notification settings - Fork 34k
gh-140715: provide support for %F token in strptime; add test for it and %T #140647
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Most changes to Python require a NEWS entry. Add one using the blurb_it web app or the blurb command-line tool. If this change has little impact on Python users, wait for a maintainer to apply the |
Misc/NEWS.d/next/Library/2025-10-27-00-13-04.gh-issue-140644.WkozE0.rst
Outdated
Show resolved
Hide resolved
|
I moved this under #140715, can you please update the blurb filename. |
Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
…kozE0.rst Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
I think this is done! |
Misc/NEWS.d/next/Library/2025-10-27-00-13-04.gh-issue-140715.WkozE0.rst
Outdated
Show resolved
Hide resolved
Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
StanFromIreland
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add them to this tuple:
Lines 361 to 363 in a2495ff
| for directive in ('a', 'A', 'b', 'B', 'c', 'd', 'H', 'I', | |
| 'j', 'm', 'M', 'p', 'S', | |
| 'U', 'w', 'W', 'x', 'X', 'y', 'Y', 'Z', '%'): |
Also, please update the docs removing the (0) note from its row.
The implementation looks good, just a few small things and this will be ready :-)
Misc/NEWS.d/next/Library/2025-10-27-00-13-04.gh-issue-140715.WkozE0.rst
Outdated
Show resolved
Hide resolved
Misc/NEWS.d/next/Library/2025-10-27-00-13-04.gh-issue-140715.WkozE0.rst
Outdated
Show resolved
Hide resolved
…kozE0.rst Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
Remove extra comments Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
shorten test name Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
shorten test function name Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
|
Thank you, @StanFromIreland, all changes have been made as requested! |
|
Thank you again, @StanFromIreland, changes have been made. |
StanFromIreland
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Last few things.
style alignment, add space after function arguments as requested Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
style alignment, add space after function arguments as requested Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
fix punctuation in docs, as requested Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
fix punctuation in docs, as requested Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
style alignment, space after functional argument, as requested Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
style alignment, space after functional argument, as requested Co-authored-by: Stan Ulbrych <89152624+StanFromIreland@users.noreply.github.com>
…p.) and use `self.theclass.strptime`
|
Thanks, @StanFromIreland, the requested changes were implemented! |
StanFromIreland
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Thanks @jyalim, and congratulations for your first PR! Do you want to open a PR for one of the other format codes? |
Incrementally builds C99+ datetime support with addition of
%Fwith single line of code. Also adds two tests: one for%Fand another for previously supported%T.Passed all tests (especially changes to new one).
Passed patchcheck.
strptimetoken support and documentation modernization #140644str*timeto C11 format codes #140715